<html>
<head><meta charset="utf-8"><title>Tracking `I-slow` issues that are fixed by MIR inlining · t-compiler/wg-mir-opt · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/index.html">t-compiler/wg-mir-opt</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/topic/Tracking.20.60I-slow.60.20issues.20that.20are.20fixed.20by.20MIR.20inlining.html">Tracking `I-slow` issues that are fixed by MIR inlining</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="195471549"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/189540-t-compiler/wg-mir-opt/topic/Tracking%20%60I-slow%60%20issues%20that%20are%20fixed%20by%20MIR%20inlining/near/195471549" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Dylan MacKenzie (ecstatic-morse) <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/topic/Tracking.20.60I-slow.60.20issues.20that.20are.20fixed.20by.20MIR.20inlining.html#195471549">(Apr 27 2020 at 20:33)</a>:</h4>
<p>What do people think of a new GH tag for <code>I-slow</code> issues that are fixed by enabling MIR inlining? Presumably it will make closing old issues easier if/when we do some MIR inlining by default on stable.  See <a href="https://github.com/rust-lang/rust/issues/68667#issuecomment-620203064" title="https://github.com/rust-lang/rust/issues/68667#issuecomment-620203064">https://github.com/rust-lang/rust/issues/68667#issuecomment-620203064</a></p>



<a name="195471898"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/189540-t-compiler/wg-mir-opt/topic/Tracking%20%60I-slow%60%20issues%20that%20are%20fixed%20by%20MIR%20inlining/near/195471898" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Wesley Wiser <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/topic/Tracking.20.60I-slow.60.20issues.20that.20are.20fixed.20by.20MIR.20inlining.html#195471898">(Apr 27 2020 at 20:36)</a>:</h4>
<p>I'd like that! It would also be good to harvest some real world test cases from that to beef up our mir-opt tests.</p>



<a name="195472465"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/189540-t-compiler/wg-mir-opt/topic/Tracking%20%60I-slow%60%20issues%20that%20are%20fixed%20by%20MIR%20inlining/near/195472465" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Dylan MacKenzie (ecstatic-morse) <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/topic/Tracking.20.60I-slow.60.20issues.20that.20are.20fixed.20by.20MIR.20inlining.html#195472465">(Apr 27 2020 at 20:42)</a>:</h4>
<p><span class="user-mention silent" data-user-id="211727">Jonas Schievink</span> suggested using optimization-specific tags in <a href="#narrow/stream/131828-t-compiler/topic/Creating.20new.20GitHub.20tags" title="#narrow/stream/131828-t-compiler/topic/Creating.20new.20GitHub.20tags">T-compiler</a>. How does <code>A-mir-opt-inlining</code> sound?</p>



<a name="195472944"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/189540-t-compiler/wg-mir-opt/topic/Tracking%20%60I-slow%60%20issues%20that%20are%20fixed%20by%20MIR%20inlining/near/195472944" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Jonas Schievink  [he/him] <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/topic/Tracking.20.60I-slow.60.20issues.20that.20are.20fixed.20by.20MIR.20inlining.html#195472944">(Apr 27 2020 at 20:47)</a>:</h4>
<p>Seems good!</p>



<a name="195482543"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/189540-t-compiler/wg-mir-opt/topic/Tracking%20%60I-slow%60%20issues%20that%20are%20fixed%20by%20MIR%20inlining/near/195482543" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Félix Fischer <a href="https://rust-lang.github.io/zulip_archive/stream/189540-t-compiler/wg-mir-opt/topic/Tracking.20.60I-slow.60.20issues.20that.20are.20fixed.20by.20MIR.20inlining.html#195482543">(Apr 27 2020 at 22:23)</a>:</h4>
<p>This sounds like a really good idea! With Wesley we also have made some changes that will for the moment not make it to <code>mir-opt-lvl=1</code> because with current codegen, they make performance regressions. Tagging the future issue that represents our change would definitely help keep track of it.</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>